Skip to content

feat(aad): support aad_ip_ddos_statistics and aad_flow_block data sources #7623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

jinyangyang222
Copy link
Contributor

@jinyangyang222 jinyangyang222 commented Aug 22, 2025

What this PR does / why we need it:

commit1: support aad_ip_ddos_statistics data source
commit2: support aad_flow_block data source

Which issue this PR fixes:
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
fixes #xxx

Special notes for your reviewer:

Release note:


commit1: support `aad_ip_ddos_statistics` data source
commit2: support `aad_flow_block` data source

PR Checklist

  • Tests added/passed.

Due to limited testing conditions, this data source cannot be tested and the API was not successfully called.

  • Documentation updated.

  • Schema updated.

  • CheckDeleted.

    • a. During query operation (Read Context)
      aa. Resource not found
      >>>>>> Paste the screenshot here <<<<<<

    • b. During delete/disassociate/unbind operation (Delete Context)
      ba. Resource not found
      >>>>>> Paste the screenshot here <<<<<<

@baixinsui
Copy link
Contributor

LGTM

@jinyangyang222 jinyangyang222 force-pushed the feat-aad/support-ip_ddos_statistics-dataSource branch from 74bef61 to ba9fadd Compare August 22, 2025 09:29
@jinyangyang222 jinyangyang222 changed the title feat(aad): support aad_ip_ddos_statistics data source feat(aad): support aad_ip_ddos_statistics anddata source Aug 25, 2025
@jinyangyang222 jinyangyang222 changed the title feat(aad): support aad_ip_ddos_statistics anddata source feat(aad): support aad_ip_ddos_statistics and aad_flow_block data sources Aug 25, 2025
@deer-hang
Copy link
Contributor

/approve
/lgtm

@github-ci-robot github-ci-robot added LGTM /lgtm approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Aug 26, 2025
Copy link
Collaborator

@github-ci-robot github-ci-robot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has been approved by: @deer-hang

/approve
/lgtm

@github-ci-robot github-ci-robot merged commit 947979f into huaweicloud:master Aug 26, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. feature LGTM /lgtm size/L size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants